Conversation
|
I have fixed the necessary conflicts you can now approve merge on the develop branch. Thanks! |
extractor.py
Outdated
logllm/log_llm.py
Outdated
| wandb.init(project=project_name, settings=wandb.Settings(_disable_stats=True)) | ||
|
|
||
| def extract_experimental_conditions(code): | ||
| def extract_experimental_conditions_gemini(code): |
There was a problem hiding this comment.
I think you do not have to make two functions, you can have one function where you use if branch to switch openai and gemini
| {{ | ||
| "method":"str", | ||
| "dataset":"str", | ||
| "task":"str", | ||
| "accuracy":"", | ||
| "other_param_here":{ | ||
| "other_param_here":"", | ||
| "other_param_here":"", | ||
| }, | ||
| "other_param_here":"", | ||
| ... | ||
| "condition_as_natural_langauge":["Small dataset."], | ||
| "advice_to_improve_acc":["Use bigger dataset.","Use more simple model."] | ||
| }} |
There was a problem hiding this comment.
Yes please check on the suggestions I made on the issues post.
There was a problem hiding this comment.
This is an implementing feature, we can ignore for now.
There was a problem hiding this comment.
You should at least make it more clear what role intended for that query function on the logllm.
There was a problem hiding this comment.
you may not need to change this file
|
Please check this suggestion |
2 participants
Changed the generative model to Gemini
Changed the
setupconfigurationChanged the
test_loggercompatible to the gemini modelAdjusted the
query.pyfile for gemini model